Skip to content

Specifically mark native methods to keep#893

Merged
davidliu merged 2 commits intomainfrom
dl/sentry_proguard
Mar 23, 2026
Merged

Specifically mark native methods to keep#893
davidliu merged 2 commits intomainfrom
dl/sentry_proguard

Conversation

@davidliu
Copy link
Contributor

By default, android's proguard will mark all native methods as kept from obfuscation (by using getDefaultProguardFile(<default proguard filename>) in build.gradle). If this is opted out of (e.g. they want to roll more targeted rules for their own JNI code), then libwebrtc will get obfuscated improperly.

@changeset-bot
Copy link

changeset-bot bot commented Mar 23, 2026

🦋 Changeset detected

Latest commit: 62d9f3f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
client-sdk-android Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

Diffuse output:

OLD: diffuse-source-file
NEW: livekit-android-sdk-release.aar

 AAR      │ old      │ new      │ diff   
──────────┼──────────┼──────────┼────────
      jar │    4 MiB │    4 MiB │    0 B 
 manifest │  1.5 KiB │  1.5 KiB │    0 B 
 lint-jar │ 12.7 KiB │ 12.7 KiB │    0 B 
    other │  1.6 KiB │  1.8 KiB │ +206 B 
──────────┼──────────┼──────────┼────────
    total │    4 MiB │    4 MiB │ +206 B 

 JAR     │ old   │ new   │ diff      
─────────┼───────┼───────┼───────────
 classes │  2205 │  2205 │ 0 (+0 -0) 
 methods │ 42929 │ 42929 │ 0 (+0 -0) 
  fields │  8368 │  8368 │ 0 (+0 -0)
AAR
 size    │ diff   │ path           
─────────┼────────┼────────────────
 1.6 KiB │ +206 B │ ∆ proguard.txt 
─────────┼────────┼────────────────
 1.6 KiB │ +206 B │ (total)

@davidliu davidliu merged commit e535571 into main Mar 23, 2026
6 checks passed
@davidliu davidliu deleted the dl/sentry_proguard branch March 23, 2026 08:13
@davidliu davidliu mentioned this pull request Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant